Skip to content

feat: API Server - Pipeline runs - Added execution summary to the list method#96

Merged
yuechao-qin merged 1 commit intomasterfrom
ycq/api_pipeline_run_list_has_ended
Apr 27, 2026
Merged

feat: API Server - Pipeline runs - Added execution summary to the list method#96
yuechao-qin merged 1 commit intomasterfrom
ycq/api_pipeline_run_list_has_ended

Conversation

@yuechao-qin
Copy link
Copy Markdown
Collaborator

@yuechao-qin yuechao-qin commented Feb 17, 2026

TL;DR

Closes #87

Added execution summary to pipeline run responses to provide aggregated information about pipeline execution status (GET /api/pipeline_runs/)..

What changed?

  • Added execution_summary field to PipelineRunResponse dataclass
  • Renamed _get_execution_status_stats to _get_execution_stats_and_summary to return both stats and summary

How to test?

  1. Create a pipeline run with multiple nodes in different states
  2. Call the list API with include_execution_stats=True
  3. Verify the response contains both execution stats and execution summary
  4. Check that the summary correctly reports total nodes, ended nodes, and completion status

Run the new test cases in tests/test_api_server_sql.py:

pytest tests/test_api_server_sql.py

Why make this change?

This closes #87

image.png

@yuechao-qin yuechao-qin marked this pull request as ready for review February 17, 2026 17:40
@yuechao-qin yuechao-qin requested a review from Ark-kun as a code owner February 17, 2026 17:40
@yuechao-qin yuechao-qin changed the base branch from ycq/function_get_execution_status_stats to graphite-base/96 February 18, 2026 02:26
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from ea578e3 to 489c797 Compare February 18, 2026 02:26
@yuechao-qin yuechao-qin changed the base branch from graphite-base/96 to ycq/function_get_execution_status_stats February 18, 2026 02:27
@yuechao-qin yuechao-qin changed the base branch from ycq/function_get_execution_status_stats to graphite-base/96 February 20, 2026 04:27
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from 489c797 to 3df5bde Compare February 20, 2026 04:28
@yuechao-qin yuechao-qin changed the base branch from graphite-base/96 to ycq/function_get_execution_status_stats February 20, 2026 04:28
@yuechao-qin yuechao-qin force-pushed the ycq/function_get_execution_status_stats branch 2 times, most recently from e382bc8 to 5ae141c Compare February 21, 2026 02:41
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from 3df5bde to 5014f69 Compare February 21, 2026 02:41
Comment thread cloud_pipelines_backend/api_server_sql.py Outdated
Comment thread cloud_pipelines_backend/api_server_sql.py Outdated
@yuechao-qin yuechao-qin changed the base branch from ycq/function_get_execution_status_stats to graphite-base/96 April 4, 2026 04:20
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from 5014f69 to cec5214 Compare April 4, 2026 04:50
@yuechao-qin yuechao-qin changed the base branch from graphite-base/96 to ycq/function_get_execution_status_stats April 4, 2026 04:50
@yuechao-qin yuechao-qin requested a review from morgan-wowk April 4, 2026 04:51
@Ark-kun Ark-kun changed the title feat: API pipeline run list has execution summary feat: API Server - Pipeline runs - Added execution summary to the list method Apr 17, 2026
@yuechao-qin yuechao-qin force-pushed the ycq/function_get_execution_status_stats branch from 0a2d4ba to c925a0a Compare April 22, 2026 14:14
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from cec5214 to 4edaac6 Compare April 22, 2026 14:14
Copy link
Copy Markdown
Collaborator Author

yuechao-qin commented Apr 27, 2026

Merge activity

  • Apr 27, 7:45 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 27, 7:47 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 27, 7:48 PM UTC: @yuechao-qin merged this pull request with Graphite.

@yuechao-qin yuechao-qin changed the base branch from ycq/function_get_execution_status_stats to graphite-base/96 April 27, 2026 19:45
@yuechao-qin yuechao-qin changed the base branch from graphite-base/96 to master April 27, 2026 19:46
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from 4edaac6 to a2e113d Compare April 27, 2026 19:46
@yuechao-qin yuechao-qin merged commit 8aabf2d into master Apr 27, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API Server - Add has_ended to the response of PieplineRunsService.list

3 participants